Skip to content

Conversation

@aabmass
Copy link
Collaborator

@aabmass aabmass commented Nov 4, 2025

No description provided.

@aabmass aabmass force-pushed the update-adk-otel-to-cloud branch from 37b861d to 6cf35a0 Compare November 4, 2025 04:13
# For this example this would be the current directory containing main.py.
# Note: Calling this method attempts to set the global tracer provider, which has already been
# set by the setup_opentelemetry() function.
app = get_fast_api_app(
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This whole custom driver main.py is no longer needed so I deleted it. --otel_to_cloud handles it all.

Alternatively, we could leave this boilerplate and pass get_fast_api_app(..., otel_to_cloud=True) and keep some boilerplate if it's easier for snippets.

@aabmass aabmass marked this pull request as ready for review November 4, 2025 04:22
@aabmass aabmass requested a review from a team as a code owner November 4, 2025 04:22
@snippet-bot
Copy link

snippet-bot bot commented Nov 4, 2025

Here is the summary of changes.

You are about to delete 2 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@aabmass aabmass force-pushed the update-adk-otel-to-cloud branch from 6cf35a0 to 68327a2 Compare November 4, 2025 17:51
@aabmass aabmass enabled auto-merge (squash) November 4, 2025 17:51
@aabmass aabmass merged commit d72f606 into GoogleCloudPlatform:main Nov 4, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants